Hi @amommendes. Thanks for your PR.

I'm waiting for a feast-dev member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Codecov Report

Merging #1957 (a9933fd) into master (3768766) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1957   +/-   ##
=======================================
  Coverage   82.07%   82.07%           
=======================================
  Files         100      100           
  Lines        7987     7987           
=======================================
  Hits         6555     6555           
  Misses       1432     1432           

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3768766...a9933fd. Read the comment docs.

/ok-to-test

Hey @amommendes, thanks for this PR. We just fixed the issue with the linter that's been blocking all PRs for the last two days. Would you mind rebasing your changes on master and then force pushing your changes up? Thanks!

Hey @amommendes, thanks for this PR. We just fixed the issue with the linter that's been blocking all PRs for the last two days. Would you mind rebasing your changes on master and then force pushing your changes up? Thanks!

Done @felixwang9817 , tks!

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: achals, amommendes, felixwang9817

The full list of commands accepted by this bot can be found here.

The pull request process is described here